-
Notifications
You must be signed in to change notification settings - Fork 114
Removed unnecessary CSS for dashboard menu button to clean up UI #880
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@tulsishuka is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
import { useHistory } from "@docusaurus/router"; | ||
import { Home, MessageCircle, Gift, Trophy, Crown, Star, Award, Clock, Users, TrendingUp, Medal, ArrowLeft } from "lucide-react"; | ||
import "../dashboard.css"; | ||
import { add } from "date-fns"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why import this
@tulsishuka have you pulled the latest release before changing this? You need to fetch from upstream and merge then |
okay thanks |
@tulsishuka you can do this and let me know but do git fetch upstream
git checkout <you-branch-that-you-created>
git rebase upstream/main
# Resolve conflicts as they appear
git push --force-with-lease |
and tell me if any design needed |
check in with @Adez017 also mention your issue on the PR description |
@sheikhlimon thanks for jumbing in and helping other contributors. its good OSS mindset, also u ranked 1st for this week leaderboard. i will sponsor small amount via Github please enable github sponsorship. |
@sanjay-kv I’m really grateful for your offer to sponsor me. Sadly, GitHub sponsorship isn’t available in my country right now. I truly appreciate your support. It motivates me to keep contributing! |
then i can provide mentorship for free which i usually charge people. @sheikhlimon |
@tulsishuka , resolve the conflicts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assigned to fix the conflicts @sanjay-kv
@sanjay-kv That’s awesome... Getting mentorship from someone with your experience is really appreciated |
https://www.linkedin.com/in/sanjay-k-v/ you can get connected over here. :) i will guide you from here @sheikhlimon |
Description
I removed unused CSS related to the old dashboard toggle button (.dashboard-menu-btn). This cleanup improves maintainability and removes redundant styles.


and another picture
please let me know i improve further